Skip to content

Move project storage code from data_connectors to storage module - #1397

Open
eikek wants to merge 7 commits into
feature/project-storagefrom
eikek/move-to-storage-module
Open

Move project storage code from data_connectors to storage module#1397
eikek wants to merge 7 commits into
feature/project-storagefrom
eikek/move-to-storage-module

Conversation

@eikek

@eikek eikek commented Jul 31, 2026

Copy link
Copy Markdown
Member

Moves the project storage code from the data_connectors module into the storage module.

/deploy

@RenkuBot

Copy link
Copy Markdown
Contributor

You can access the deployment of this PR at https://renku-ci-ds-1397.dev.renku.ch

@eikek
eikek force-pushed the eikek/move-to-storage-module branch from 1cadb87 to 1755f91 Compare July 31, 2026 08:52
@eikek
eikek force-pushed the eikek/move-to-storage-module branch from 3f1c1c8 to 00a37a7 Compare July 31, 2026 11:21
@coveralls

coveralls commented Jul 31, 2026

Copy link
Copy Markdown

Coverage Report for CI Build 30634100763

Warning

No base build found for commit f74ba4a on feature/project-storage.
Coverage changes can't be calculated without a base build.
If a base build is processing, this comment will update automatically when it completes.

Coverage: 85.976%

Details

  • Patch coverage: 41 uncovered changes across 6 files (531 of 572 lines covered, 92.83%).

Uncovered Changes

File Changed Covered %
components/renku_data_services/storage/db.py 195 165 84.62%
components/renku_data_services/storage/blueprints.py 139 135 97.12%
components/renku_data_services/storage/core.py 40 36 90.0%
components/renku_data_services/storage/config.py 23 22 95.65%
components/renku_data_services/storage/models.py 63 62 98.41%
components/renku_data_services/storage/orm.py 44 43 97.73%
Total (18 files) 572 531 92.83%

Coverage Regressions

Requires a base build to compare against. How to fix this →


Coverage Stats

Coverage Status
Relevant Lines: 32438
Covered Lines: 27889
Line Coverage: 85.98%
Coverage Strength: 1.49 hits per line

💛 - Coveralls

@eikek
eikek marked this pull request as ready for review July 31, 2026 12:36
@eikek
eikek requested review from a team, SalimKayal and sgaist as code owners July 31, 2026 12:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants